window: Restructure gtk_window_destroy()

The previous code was unreffing the window twice, which caused problems
during dialog destruction. Move to g_list_store_find instead of
iterating manually.

ref() the window before and unref() after. g_list_store_remove will
actually unref() the window, since the toplevel_list owns its own
reference.

Fixes #2741
Fixes #2742

main: Inline propagate_{down,up} in propagate_internal()

Don't have to worry about anyone calling the up/down variants on their
own this way.

window: Create constraint solver on demand

widget: Make 1 if out of 2 ifs

paned: Fix up the docs once more

cssvalue: Add class check to transition

In b25f93e24c78c639b4fd869d4e42ba9161781c12 we removed the code
comparing the transition functions, but we didn't add any other check
back.

Fixes #2740

gl renderer: Fix nested rounded clip rendering

If the inner clip intersects with the corners of the outer clip, we
potentially need a texture. We should add more fine-grained checks for
this in the future though.

Test case included.

node-editor: Monitor file and reload on changes

widget: Move adjusted size calculation below early-out goto

widget: Remove size allocate signal enum member

renderboder: Ignore transparent outlines

Merge branch 'matthiasc/for-master' into 'master'

x11: Avoid a critical in clipboard data transfer

Closes #2736

See merge request GNOME/gtk!1887

x11: Avoid a critical in clipboard data transfer

The handler we are calling here asserts that it gets
an interned string as mime_type. So give it one.

Fixes: #2736

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1885

Merge branch 'tests-cleanup' into 'master'

Tests cleanup

See merge request GNOME/gtk!1884

paned: Don't ignore the type attribute

We were meaning to only handle <child> here for
compatibility, not <child type="somethingelse">.

paned: Drop the Private struct

tests: Drop the visuals test

This was just a different way to load a ui file
and try it with dark mode. This can be done
with gtk-builder-tool preview and GTK_THEME.

tests: Move testcairo into gtk4-demo

Replace the boring checkboard pattern with this
exciting drawing, and save one test binary at the
same time!

Drop the gtkgears test

This has recently been moved to gtk4-demo.
No need to duplicate it.

Move print-editor to demos

It is too demo-esque for languishing in tests/.

NEWS: Updates

Merge branch 'matthiasc/for-master' into 'master'

printoperation-unix: Fix a use-after-free

See merge request GNOME/gtk!1883

Merge branch 'docs-fixes' into 'master'

Docs fixes

See merge request GNOME/gtk!1882

printoperation-unix: Fix a use-after-free

It turns out that finish_print destroys the
PrintResponseData, so we can't access it afterwards.

docs: Add annotations for GtkPaned new getters

Which means also adding gtk-doc stanzas.

docs: Remove unnecessary SECTION

The GdkDragSurface definitions fall under the "dnd" section.

docs: Remove redundant SECTION

We already have a SECTION stanza for gtk-doc.

Merge branch 'wip/carlosg/broken-grabs' into 'master'

Fixes to broken grabs

Closes #2665

See merge request GNOME/gtk!1879

Merge branch 'matthiasc/for-master' into 'master'

window: Hide windows on destroy

See merge request GNOME/gtk!1881

gtkmain: Release the active state on GDK_GRAB_BROKEN

This is handled internally here, but misses broken grabs. The active
state should be dropped just as if the button were released.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/2665

window: Hide windows on destroy

I thought I could get away with just unrealizing the
window, but it turns out that gtk_window_hide() is the
place where we remove grabs when a modal dialog goes
away, so we ended up with stuck grabs.

Merge branch 'matthiasc/for-master' into 'master'

Documentation work

See merge request GNOME/gtk!1880

Merge branch 'ebassi/run-dialog-run' into 'master'

Remove gtk_dialog_run() and friends

Closes #371

See merge request GNOME/gtk!1801

Do not release the GFile prematurely

Otherwise we won't be able to access it to get the URI for the
GtkPrinterOption.

This fixes a regression introduced in commit 5f070ff2333.

docs: Mention blocking functions in the migration guide

Link to how to make a dialog modal, and to the response signal.

Remove gtk_dialog_run()

Nested main loops are bad, as they introduce layers of complexity caused
by the potential re-entrancy in the case of multiple event sources, like
IPC, threads, etc. Additionally, the programming model they provide—stop
the world while spinning a new loop—does not conform to the event-driven
model employed by GTK.

Remove gtk_dialog_run() from GtkPrintUnixDialog

Replace it with an explicit nested main loop, as we need to block the
signal handler currently being emitted depending on the response of the
overwrite confirmation dialog.

This is a bit of a hack, and the only reason we need it is that the
print dialog will load the last used path as the output file name, when
printing to a file; this means that, in theory, it would be possible to
press Print without selecting a file, and accidentally overwriting an
existing file.

It would be much simpler if we did not store the last used path, and
always explicitly asked the user to select a file; this would avoid
destructive actions, and would allow us to rely on the overwrite
confirmation dialog right inside the file chooser.

Remove unnecessary deprecation pragmas

The gtk_window_present() function is not deprecated.

Drop gtk_dialog_run() from GtkPrintOperationUnix

We still provide a blocking API, but we should strongly reconsider it.

tests: Remove gtk_dialog_run()

Either use the "response" signal for dialogs that are already modal, or
use an explicit nested loop for tests that rely on the response id being
available in sequence.

Drop gtk_dialog_run() from GtkFileChooserWidget

The various dialogs we use inside the file chooser are modal already,
and do no need a nested loop.

docs: Remove use of gtk_dialog_run()

Direct people to use GTK_DIALOG_MODAL and the "response" signal instead
of nested main loops.

Remove gtk_native_dialog_run()

Nested main loops are bad, as they introduce layers of complexity caused
by the potential re-entrancy in the case of multiple event sources, like
IPC, threads, etc. Additionally, the programming model they provide—stop
the world while spinning a new loop—does not conform to the event-driven
model employed by GTK.

docs: Remove use of gtk_native_dialog_run() from examples

Use the "response" signal instead.

Remove gtk_dialog_run() from GtkMountOperation

gtkmain: Propagate GDK_GRAB_BROKEN events to/from current focus

Let keyboard/pointer paths handle their own events, and find the
current focus. The event will be propagated through instead of
being just emitted on the toplevel.

This makes it handled throughout all the gestures that want to
know about it.

gdk/wayland: Drop explicit ungrab after surface move/resize

We now break grabs properly if receiving a .leave event with
active buttons. This is not necessary anymore.

gdk/wayland: Ensure .leave events with active buttons breaks grabs

Instead of silently ending up the grab, make it sure that we emit
a GDK_GRAB_BROKEN event if needed.

gdk/wayland: Keep track of implicit grabs

Tracking of those broke sometime along the gdk cleanups, so we
started missing some GDK_GRAB_BROKEN events from being emitted
(eg. after a button press/implicit grab triggers an active grab).

Implicit grabs are only added if there's no prior grab (either
implicit through other button presses, or explicit), in order to
keep accounting correct, make those prevail.

gtktexthandle: Set css name on GtkWidgetClass

Handles are their own widget, special names are not necessary
anymore.

demos: Remove gtk_dialog_run()

Use modal dialogs and the "response" signal.

docs: Updates to the migration guide

Remove some duplicated content, reorder things
to make more sense, and tweak the wording.

docs: Tweaks

Replace some occurrences of GTK+ by GTK.

docs: Updates to the common questions

Refresh the answers to some common questions.

docs: Don't point at mailing lists

Replace references to the mailing lists by
discourse + irc.

docs: Rearrange the introduction some more

Fine-tune some wording, and move the Custom Drawing
example earlier.

Merge branch 'kill-containers' into 'master'

Kill containers

See merge request GNOME/gtk!1866

docs: Update migration guide

Add sections about GtkBin, GtkContainer and gtk_widget_destroy().

dialog: Firm up handling of action widgets

It is unreliable to use the widget dom api to locate
action widgets. For example in a headerbar, they might
be deeper in the hierarchy, with boxes in between.
Therefore, make GtkDialog keep a list of action widgets,
and use that when operating on action widgets.

dialog: Remove an unused struct

Drop GtkContainer and its accessible implementation

It is no longer used.

testsuite: Drop a container api use

Drop gtkcontainer.h includes

These includes are unused.

box: Derive from GtkWidget

assistant: Remove code that doesn't work anymore

When GtkContainer goes away, we don't have
an "add" signal anymore.

dialog: Remove code that doesn't work anymore

When GtkContainer goes away, we don't have an
"add" signal anymore.

shortcutssection: Drop container bits

When GtkBox stops being a container, GtkShortcutsSection
will also no longer be a container. So, stop overriding
container vfuncs, and instead add a buildable implementation
that does the right thing.

shortcutsgroup: Drop container bits

When GtkBox stops being a container, GtkShortcutsGroup
will also no longer be a container. So, stop overriding
container vfuncs, and instead add a buildable implementation
that does the right thing.

Replace most remaining uses of container api

These are all on GtkBox or enumerating children.

box: Add gtk_box_append/prepend/remove

Add replacement api for gtk_container_add/remove.

notebook: Derive from GtkWidget

Stop using container api on GtkNotebook

headerbar: Derive from GtkWidget

Stop using container api on GtkHeaderBar

headerbar: Add gtk_header_bar_remove

This is a replacement for gtk_container_remove.

listbox: Derive from GtkWidget

testsuite: Fix listbox tests

When moving from gtk_container_forall to the widget dom
api, we are now iterating over all children of the listbox,
including headers, separators, etc. So, skip everything
that is not a listboxrow, to make the tests work again.

testsuite: Redo listbox sort test

This test was relying on gtk_container_forall returning
the visual (ie sorted) order of children, while iterating
with the widget dom api gives the insertion order.

Instead of using gtk_container_forall, use
gtk_list_box_row_get_index to reconstruct the visual
order.

Don't use container api on GtkListBox

listbox: Add gtk_list_box_remove

This is a gtk_container_remove replacement.

buildable: Cosmetics

Don't refer to GtkContainer in the docs.

customlayout: Cosmetics

Don't refer to GtkContainer in the docs.

textviewchild: Derive from GtkWidget

GtkContainer is going away.

text: Fix popover use

There were some leftover gtk_container_add calls here
on popovers on buttons, but these are no longer containers.

textview: Fix popover use

There were some leftover gtk_container_add calls here
on popovers on buttons, but these are no longer containers.

flowbox: Derive from GtkWidget

Stop using container api on GtkFlowBox

GtkContainer is going away.

flowbox: Add gtk_flow_box_remove

This is the replacement for gtk_container_remove.

stack: Derive from GtkWidget

treepopover: Stop using container api

shortcutswindow: Stop using container api

shortcutssection: Stop using container api

Don't use container api on GtkStack

stack: Add gtk_stack_remove

This is a replacement for gtk_container_remove.

fixed: Derive from GtkWidget

Stop using container api on GtkFixed

GtkContainer is going away.

fixed: Add gtk_fixed_remove

This is the replacement for gtk_container_reomve.

grid: Derive from GtkWidget

GtkContainer is going away.

grid: Remove a test for grid container functionality

This test was specifically testing how gtk_container_add
behaves on grids. Well, it doesn't anymore.